Skip to content

feat: add registry store for tracking agent registrations#104

Merged
jithinkunjachan merged 3 commits intomainfrom
feat/add-registraion-store
Apr 13, 2026
Merged

feat: add registry store for tracking agent registrations#104
jithinkunjachan merged 3 commits intomainfrom
feat/add-registraion-store

Conversation

@jithinkunjachan
Copy link
Copy Markdown
Contributor

@jithinkunjachan jithinkunjachan commented Apr 10, 2026

Introduce a PostgreSQL-backed registry store that records agent instances and their heartbeat status during registration.
The agent API now requires an X-Agent-Id header and upserts registry entries on each registration request.

Introduce a PostgreSQL-backed registry store that records agent
instances and their heartbeat status during registration. The agent
API now requires an X-Agent-Id header and upserts registry entries
on each registration request.

# Conflicts:
#	pkg/api/agents/client.go
@jithinkunjachan jithinkunjachan force-pushed the feat/add-registraion-store branch from 1ec3e72 to 404b8b6 Compare April 10, 2026 07:24
@jithinkunjachan jithinkunjachan self-assigned this Apr 10, 2026
@jithinkunjachan jithinkunjachan marked this pull request as ready for review April 10, 2026 07:24
@push-tags-from-workflow push-tags-from-workflow bot added dependencies Pull requests that update a dependency file tests feature labels Apr 10, 2026
Copy link
Copy Markdown

@apatsap apatsap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Just some naming remarks

Comment thread pkg/store/sql/registry_test.go Outdated
Comment thread internal/spec/registry.go Outdated
Comment thread internal/spec/registry.go Outdated
Comment thread pkg/store/registry.go Outdated
@jithinkunjachan jithinkunjachan marked this pull request as draft April 10, 2026 12:49
@jithinkunjachan jithinkunjachan requested a review from apatsap April 10, 2026 13:32
@jithinkunjachan jithinkunjachan marked this pull request as ready for review April 10, 2026 13:32
Copy link
Copy Markdown

@apatsap apatsap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Turned out nicely!

Copy link
Copy Markdown
Contributor

@fabenan-f fabenan-f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@jithinkunjachan jithinkunjachan merged commit 0a509eb into main Apr 13, 2026
5 checks passed
@jithinkunjachan jithinkunjachan deleted the feat/add-registraion-store branch April 13, 2026 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file feature tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants